fix(icons): repair broken integration icon rendering#4810
Conversation
Two distinct bugs left integration icons broken on the /integrations page (visible at 32-40px, hidden at the toolbar's 16px): 1. Corrupted SVG paths (Notion, Greptile, Granola, Calendly, Grafana, Bedrock): over-minified data dropped elliptical-arc flag digits (e.g. `A1 1 0 5.9 7` instead of `A1 1 0 0 0 5.9 7`); Granola's cubic stream was truncated. Browsers abort path parsing at the first invalid arc flag, so each rendered as a fragment or blank. Replaced with correct path data from canonical sources, preserving each icon's existing fill/gradient and bgColor. 2. Invisible glyph (Bright Data): its icon uses fill='currentColor' but bgColor was '#FFFFFF', and every surface forces text-white on the glyph - white-on-white. Changed bgColor to Bright Data's brand blue (#3d7ffc) so the white glyph reads, matching the white-glyph-on-brand-chip convention. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Several icons (Notion, Grafana, Calendly, Granola, Greptile, and the gradient Bedrock-style glyph) had invalid or truncated Bright Data was effectively invisible because Reviewed by Cursor Bugbot for commit b64089d. Configure here. |
Greptile SummaryThis PR fixes rendering regressions in 7 integration icons by replacing over-minified or truncated SVG path data with correct path data from canonical sources, and fixes the Bright Data icon's invisible white-on-white rendering by changing its chip background to the brand blue.
Confidence Score: 5/5Safe to merge — all changes are isolated SVG path data and colour corrections with no logic, API, or data-model impact. Every changed line is either SVG path geometry, a viewBox attribute, a gradient coordinate, or a bgColor string. Both apps/sim and apps/docs icon files were updated in lockstep. The Bright Data bgColor is updated consistently in both the block definition and the integrations JSON. No shared utilities, state, or API contracts were touched. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Icon broken / invisible on /integrations page] --> B{Root cause}
B --> C[SVG path data\nover-minified / truncated]
B --> D[BrightData: white glyph\non white background]
C --> E[Replace path data with\ncorrect canonical source]
D --> F[Change bgColor\n#FFFFFF → #3d7ffc]
E --> G[apps/sim/components/icons.tsx\nNotionIcon, GrafanaIcon,\nCalendlyIcon, GranolaIcon,\nGreptileIcon, BedrockIcon]
E --> H[apps/docs/components/icons.tsx\nIdentical changes - kept in sync]
F --> I[apps/sim/blocks/blocks/brightdata.ts]
F --> J[apps/sim/app/landing/integrations/data/integrations.json]
G --> K[Icons render correctly at 16px & 40px]
H --> K
I --> K
J --> K
Reviews (2): Last reviewed commit: "fix(icons): restore Calendly dual-tone b..." | Re-trigger Greptile |
Addresses review feedback: the previous fix replaced the broken Calendly icon with a monochrome #006BFF path, dropping the cyan #0ae8f0 accent from the original dual-tone mark. Restored the two-tone logo (blue + cyan) using clean, valid path data, cropped to a tight square viewBox so it fills the chip. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit b64089d. Configure here.
- Zoom: glyph was blue-on-blue (#0B5CFF on #2D8CFF chip); switched to currentColor so it renders as a white glyph on the blue chip. - Quiver: chip bgColor #000000 -> #FFFFFF to match the icon's near-white box, and enlarged the mark slightly (viewBox crop). - Enlarged (tightened viewBox, verified no clipping): RevenueCat, Prospeo, Granola, Firecrawl, Enrich.so, and the AWS icons (RDS, DynamoDB, SQS, CloudFormation, Athena, CloudWatch, SES, Bedrock, S3). - ZoomInfo left unchanged: it is a full red rounded-square logo that already fills its frame, so a crop would clip it. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- Bright Data: replaced the flame glyph with the official two-tone 'bright data' wordmark (provided asset), centered in a symmetric viewBox. Reverted the chip bgColor from #3d7ffc to #FFFFFF since the blue wordmark is invisible on a blue chip (the wordmark is designed for a light background). - Circleback: a minifier had rounded the pattern's image scale to scale(0), collapsing the embedded logo to zero size (invisible). Restored the correct scale (1/280 = 0.00357142857) so the C. mark renders. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Reflects the Quiver bgColor change (#000000 -> #FFFFFF) in the docs block info card. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
… Zoom - Enlarged (tighter viewBox, render-verified, no clipping): Cloudflare, Dagster, and the red AWS icons AWS IAM, Identity Center, Secrets Manager, SES, STS. Identity Center was anomalously small (filled ~32% of its frame); the group is now sized consistently (~80% fill). - Zoom: the camera lens triangle was still #0B5CFF (blue-on-blue); switched it to currentColor so the whole camera renders white on the blue chip. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Merges the separate Start/Get Individual Reveal operations into one Individual Reveal operation in the Wiza docs and integrations data (operationCount 5 -> 4). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ill) Bring RDS, DynamoDB, SQS, CloudFormation, Athena, CloudWatch and S3 up to the same ~80% fill as the AWS IAM/Identity Center/Secrets Manager/SES/STS group, so all AWS icons are visually consistent. Bedrock left as-is (already ~92% fill). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- Bright Data: the full 'bright data' wordmark was illegible at chip size. Replaced with just the flame-'i' brand mark (blue #4280f6 on the white chip), centered. - ZoomInfo: cropped the viewBox toward the white 'Zi' so it's larger; the red rounded-square background still fills the chip. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The falcon mark sat small in its chip because the icon used a wide 768x500 viewBox (letterboxed in the square chip). Switched to a square viewBox centered on the mark so it fills ~80%, consistent with the other icons. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Summary
A1 1 0 5.9 7instead ofA1 1 0 0 0 5.9 7; Granola's cubic stream was truncated), so browsers aborted path parsing at the first bad flag. Replaced with correct path data from canonical sources, preserving each icon's existing fill/gradient and bgColor.fill='currentColor'on a#FFFFFFchip while every surface forcestext-white(white-on-white). Changed its bgColor to Bright Data's brand blue (#3d7ffc) so the white glyph reads.Type of Change
Testing
Rendered each affected icon at 16px and 40px on its chip in headless Chromium to confirm complete, visible logos. Verified only these icon definitions changed and Calendly's white chip was left untouched.
Checklist